Skip to content

Enhance TP/SL input UX#144

Open
Jiangzhewen wants to merge 1 commit into
capofficial:mainfrom
Jiangzhewen:enhanced-tpsl-inputs
Open

Enhance TP/SL input UX#144
Jiangzhewen wants to merge 1 commit into
capofficial:mainfrom
Jiangzhewen:enhanced-tpsl-inputs

Conversation

@Jiangzhewen
Copy link
Copy Markdown

Summary

  • adds quick TP/SL price-move buttons for +1%, +2%, and +5% targets
  • lets users edit TP/SL by target price, P/L percent, or P/L amount
  • keeps the three TP/SL input modes synchronized for both long and short order directions

Validation

  • npm run build
  • Playwright local smoke test on http://127.0.0.1:5174/trade/ETH-USD with the country overlay hidden only for local testing:
    • desktop and 390px mobile layout snapshots for expanded TP/SL controls
    • Size=1, Price=1000, Profit (%)=10 updates TP Price to 1002 and Profit (ETH) to 0.002
    • Profit (ETH)=0.004 updates Profit (%) to 20 and TP Price to 1004

Closes #17.

@Jiangzhewen
Copy link
Copy Markdown
Author

Jiangzhewen commented May 13, 2026

Local validation is passing: npm run build completes successfully. The only failing check is the repository-level Vercel integration returning "Account is blocked", which is the same upstream Vercel account issue currently affecting other open PRs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Enhanced Take-profit and Stop-loss inputs

1 participant